Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/understand sentiment parallel do #7994

Merged

Conversation

reyoung
Copy link
Collaborator

@reyoung reyoung commented Jan 31, 2018

This PR is a sub-job of #7872

Depends on #7974, #7970

@reyoung reyoung requested a review from JiayiFeng February 1, 2018 05:02
@dzhwinter
Copy link
Contributor

fix the CI failure in #8149

@dzhwinter dzhwinter mentioned this pull request Feb 6, 2018
@wangkuiyi
Copy link
Collaborator

I see the designated reviewer @JiayiFeng might be on the holiday. I am approving and merging this PR; otherwise, I am afraid that it will be leaving here for two weeks and get conflicts with other merges.

@wangkuiyi wangkuiyi merged commit 89ead8d into PaddlePaddle:develop Feb 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants